This looks good to me, but I hope we could get unit tests added for any new additions.

Though we might not be able to easily test mdls consistently, I think we could at least export the deserialize function (if not also to_js_type, though first converting to CamelCase) and checking that.

Would you mind adding a unit test--maybe Mocha--for your addition? With some testing framework in place, we can then sanity check with any new modifications (and also try to eventually add tests to cover other supported parsing scenarios).

I'm going to second the call for tests please.e

@avimak just checking in, do you intend to add tests?

sorry, can't find the time

The "Now handling parenthesis in file names" commit only selectively escapes file names. Instead of escaping the file name it should just be wrapped in quotes, eg:

exec('mdls ' + (args ? args + ' ' : '') + `"${file}"`, function(err, raw_data) {

I have this adjusted on my fork at https://github.com/pdenya/node-mdls. I merged the rest of the changes from this PR and I'm also not planning to add tests so didn't think it was worth submitting another PR against.